-
Notifications
You must be signed in to change notification settings - Fork 4.6k
[Mini from mini] v2-v4 to v6 full recipe #49098
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…o use original packedPFCandidates collection in miniAODFromMiniAOD_customizeCommon()
…ith cms-merge-topic
|
cms-bot internal usage |
|
A new Pull Request was created by @vlimant for master. It involves the following packages:
@cmsbuild, @ctarricone, @davidlange6, @fabiocos, @ftenchini, @ftorrresd, @gabrielmscampos, @hqucms, @jfernan2, @mandrenguyen, @nothingface0, @rseidita can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
enable nano |
|
please test |
|
@vlimant , bot das cache for 3 failing relvals is now updated. As I wrote earlier bot only force update its cache after 7 days if it already has a valid result (non-empty) . Anyway, I have forced run das-query jenkins job to update cache for these 3 relvals. I have now restarted failing jobs to use the newer cache, hopefully things should work this time |
|
thanks @smuzaffar |
@smuzaffar, thanks! Since we are here, something I don't understand yet is how the original DAS cached result was non-empty in the first place, given the (I tested an analogous das query locally, and it would give no result) |
|
@AdrianoDee , In past (many many years ago) we wanted IB's relvals to access only data at CERN, so we had added das-query cache script works on the above assumption. So if it can not find any data with So for issue we have seen here was that though das was returning a list of files but they were not available on disk or not accessible via AAA. So cmsRun was not able to read those files and bot was not able to cache the root files. |
|
by the way, all 3 failing workflows now passed |
|
Clear, thanks a lot. |
|
+1 Size: This PR adds an extra 24KB to repository Comparison SummarySummary:
Nano size comparison Summary:
NANO Comparison SummaryThe workflows 2500.0204, 2500.0203, 2500.0202, 2500.0201, 2500.0003, 2500.0002 have different files in step1_dasquery.log than the ones found in the baseline. You may want to check and retrigger the tests if necessary. You can check it in the "files" directory in the results of the comparisons Summary:
|
|
@cms-sw/xpog-l2 @cms-sw/pdmv-l2 : please go ahead and review/sign so that we get it in the 16.0 dev cycle, and add work on top to fix the nano vs nano discrepancies |
|
+Upgrade |
|
+pdmv |
|
+xpog |
|
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @mandrenguyen, @sextonkennedy, @ftenchini (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 |
PR description:
this PR builds on top of #48988 with the addition of the cmsDriver integration
the mini-from-mini is configured with:
cmsDriver.py step2 -s PAT:Configuration/StandardSequences/REMINI_cff.patAlgosToolsTask --era Run3 -n 100 --process REMINI --conditions auto:phase1_2024_realistic --mc --scenario pp --eventcontent MINIAODSIM --datatier MINIAODSIM --filein /store/mc/Run3Summer23MiniAODv4/TTtoLNu2Q_TuneCP5_13p6TeV_powheg-pythia8/MINIAODSIM/130X_mcRun3_2023_realistic_v14-v2/2520000/373b1353-c6b4-4ec0-bc13-b5c0eff8c189.root --fileout file:step2.rootsuggestions to a different syntax are welcome, and I'll update the PR
PR validation:
the limited matrix ran to completion, with some unrelated errors related to file access.
If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:
Backports to 15.1 and 15.0 are expected